chore: updating code owners to team - #54

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.20%. Comparing base (55de008) to head (c820ef3).
⚠️ Report is 83 commits behind head on main.

@@            Coverage Diff            @@
##               main      #54   +/-   ##
=========================================
  Coverage     69.20%   69.20%           
  Complexity      106      106           
=========================================
  Files            15       15           
  Lines           565      565           
  Branches         33       33           
=========================================
  Hits            391      391           
  Misses          154      154           
  Partials         20       20           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

Unit Test Results

  9 files  ±0    9 suites  ±0   8s ⏱️ ±0s
31 tests ±0  31 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit c820ef3. ± Comparison against base commit 55de008.

I don't have strong feelings either way, but I think I recall @skjindal93 had moved some repos away from team ownership, so might want to discuss that first.

Team base approach makes it easy to add/remove members, isn't it? Or that's not the case?

I don't have strong feelings either way, but I think I recall @skjindal93 had moved some repos away from team ownership, so might want to discuss that first.

Team base approach makes it easy to add/remove members, isn't it? Or that's not the case?

Just that team based approach needs admin to add new code owners to the team

It becomes easy otherwise to update code owners, if they are just listed out in the CODEOWNERS file. Anyone can raise a PR to add them

Just that team based approach needs admin to add new code owners to the team

Not a global or repo admin though, a team maintainer. So if that's the approach we want, we could choose to make all (or just some) members of each team as maintainers.

Just that team based approach needs admin to add new code owners to the team

Not a global or repo admin though, a team maintainer. So if that's the approach we want, we could choose to make all (or just some) members of each team as maintainers.

I am fine with either approach, as long there is an easy way to add more code owners, and seems like there is. Though, it also becomes easy to see the enumerated code reviewers in a pull request, rather than clicking on the user group and inspecting the members